Skip to content

config: do not leak file descriptor when reading config#425

Merged
zippolyte merged 1 commit intoDataDog:masterfrom
jd:fix-file-not-closed
Sep 19, 2019
Merged

config: do not leak file descriptor when reading config#425
zippolyte merged 1 commit intoDataDog:masterfrom
jd:fix-file-not-closed

Conversation

@jd
Copy link
Copy Markdown
Contributor

@jd jd commented Aug 13, 2019

Makes sure that the config file descriptor is closed once read.

Makes sure that the config file descriptor is closed once read.
@jd jd requested a review from a team as a code owner August 13, 2019 09:59
Copy link
Copy Markdown
Contributor

@zippolyte zippolyte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@jd
Copy link
Copy Markdown
Contributor Author

jd commented Sep 19, 2019

Any reason for this not to be merged already?

@zippolyte zippolyte merged commit 7c96553 into DataDog:master Sep 19, 2019
dabcoder pushed a commit to dabcoder/datadogpy that referenced this pull request Oct 25, 2019
Makes sure that the config file descriptor is closed once read.
dabcoder pushed a commit to dabcoder/datadogpy that referenced this pull request Nov 28, 2019
Makes sure that the config file descriptor is closed once read.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants